Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENVS] Envs updates #63

Open
wants to merge 100 commits into
base: main
Choose a base branch
from

Conversation

samuelmyoussef
Copy link
Member

Updates to the environments based on the no-server implementation #62

…ofaDefrost#62)

* change abstract class to no server

* add server class

* modify viewer

* modify simulate & goal initializtion

* simulation init and reset

* remove unneeded goal and reward update and unused reward update param

* add general methods to init and randomize states and goal

* no server

* class instantiation

* remove unused goal components

* add some default configs

* initialize states

* Update pole angle from position directly instead of calculation and decrease max angle allowed

* goal init

* remove non-overriden methods

* separate AbstractEnv and ServerEnv into two files
@alxbilger
Copy link
Member

Hi @samuelmyoussef, would you have the time to solve the conflicts?

@samuelmyoussef
Copy link
Member Author

Yes, I can do it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants